Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX : Hidden dropdown download link in project #32102

Merged

Conversation

atm-irvine
Copy link
Contributor

FIX|Fix Hidden dropdown download link in project

I removed the tdoverflowmax class because it contains an "overflow: hidden;" that hide the dropdown of the download button and prevent access to menu when you want to download or preview the PDF.

Before :
image

Now:
image

@@ -1288,7 +1288,7 @@
print "</td>\n";

// Ref
print '<td class="left nowraponall tdoverflowmax250">';
print '<td class="left nowraponall">';
Copy link
Member

@eldy eldy Nov 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need the tdoverflowmax250 for the case of object that do have a download arrwo and output a very long ref including a text (like loan , tasks).
The ref and is combo must alwyas be in a separated column so i suggest to split the td in 2. The first one with ref and link will be without the tdoverflow. The second that contains the very long text will have the tdoverflowmax250.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have put a table inside the td to prevent adding another td because the table head would not be right.

@eldy eldy added Discussion Some questions or discussions are opened and wait answers of author or other people to be processed PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) labels Nov 27, 2024
@atm-irvine
Copy link
Contributor Author

@eldy Is it ok for you?

@eldy eldy merged commit f527940 into Dolibarr:20.0 Dec 19, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants